Skip to content

feat: accept CCCO fixpack 1.2.2.3 - #212

Merged
ANJANA-A-R-K merged 2 commits into
mainfrom
Feature/ccco-fixpack-1.2.2.3
Nov 12, 2026
Merged

ANJANA-A-R-K merged 2 commits into
mainfrom
Feature/ccco-fixpack-1.2.2.3

Conversation

@ANJANA-A-R-K

Copy link
Copy Markdown
Contributor

Description

  • Add 1.2.2.3 to the CCCO versions listed in docs/README.md
  • Cover fixpack 1.2.2.3 in TestProcess_CCCO_ValidCertVersion
  • Requires contract-go v3.5.0 (already in go.mod), which adds 1.2.2.3

Related Issue

Fixes #

Type of Change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update
  • Code refactor (no functional changes)
  • CI/CD or build changes

Target Platform

  • HPVS (IBM Hyper Protect Virtual Servers for VPC)
  • CCRT (IBM Confidential Computing Container Runtime)
  • CCRV (IBM Confidential Computing Container Runtime for Red Hat Virtualization Solutions)
  • CCCO (IBM Confidential Computing Containers for Red Hat OpenShift Container Platform)
  • All platforms
  • Not platform-specific

Testing

  • make test passes
  • make fmt applied (no formatting changes needed)
  • New tests added for new functionality (if applicable)

Checklist

  • My code follows the project's coding standards
  • I have performed a self-review of my code
  • I have added/updated GoDoc comments for public functions
  • I have updated documentation (README, docs/README.md) if needed
  • All new and existing tests pass (make test)
  • I have verified there are no breaking changes (or documented them above)

- Add 1.2.2.3 to the CCCO versions listed in docs/README.md
- Cover fixpack 1.2.2.3 in TestProcess_CCCO_ValidCertVersion
- Requires contract-go v3.5.0 (already in go.mod), which adds 1.2.2.3

@Rohit-Singh43-1 Rohit-Singh43-1 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just basic query .

Comment thread lib/encryptString/encryptString_test.go Outdated
// TestProcess_CCCO_ValidCertVersion tests that declared CCCO mod releases and fixpacks are accepted
func TestProcess_CCCO_ValidCertVersion(t *testing.T) {
for _, version := range []string{"1.2.2", "1.2.2.1", "1.2.1"} {
for _, version := range []string{"1.2.2", "1.2.2.1", "1.2.2.3", "1.2.1"} {

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @ANJANA-A-R-K for changes , Just for my understanding should be mentioned 1.2.2.2 fix pack as well in this array ?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes, included

@ANJANA-A-R-K
ANJANA-A-R-K merged commit 15ac644 into main Nov 12, 2026
9 checks passed
@github-actions

Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 1.48.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants